Skip to content

mini-yetus: fix Docker image tag to match CI#5679

Merged
rene merged 1 commit intolf-edge:masterfrom
rucoder:rucoder/mini-yetus-update
Mar 16, 2026
Merged

mini-yetus: fix Docker image tag to match CI#5679
rene merged 1 commit intolf-edge:masterfrom
rucoder:rucoder/mini-yetus-update

Conversation

@rucoder
Copy link
Contributor

@rucoder rucoder commented Mar 15, 2026

Description

tools/mini-yetus.sh referenced lfedge/eve-yetus:0.15.10-eve-1 which does not exist on Docker Hub. The Makefile correctly uses 0.15.1-eve-1. This mismatch caused make mini-yetus to silently fail: the Docker container never started, no results file was produced, yet the script exited 0 — masking the failure entirely.

Align the tag with the Makefile so the local linter actually runs.

How to test and validate this PR

  1. Run make MYETUS_VERBOSE=Y mini-yetus on a branch with known lint issues.
  2. Before the fix: the command prints sed: can't read .../results-full.txt and reports nothing (Docker image not found, silently swallowed).
  3. After the fix: the command correctly runs the linter container and reports any lint findings.

Changelog notes

No user-facing changes.

PR Backports

  • 16.0-stable: No, tooling-only change.
  • 14.5-stable: No, tooling-only change.
  • 13.4-stable: No, tooling-only change.

Checklist

  • I've provided a proper description

  • I've added the proper documentation

  • I've tested my PR on amd64 device

  • I've tested my PR on arm64 device

  • I've written the test verification instructions

  • I've set the proper labels to this PR

  • I've checked the boxes above, or I've provided a good reason why I didn't
    check them.

tools/mini-yetus.sh referenced lfedge/eve-yetus:0.15.10-eve-1 which
does not exist on Docker Hub. The Makefile correctly uses 0.15.1-eve-1.
This mismatch caused 'make mini-yetus' to silently fail: the Docker
container never started, no results file was produced, yet the script
exited 0 — masking the failure entirely.

Align the tag with the Makefile so the local linter actually runs.

Signed-off-by: Mikhail Malyshev <mike.malyshev@gmail.com>
@rucoder rucoder requested a review from rene March 15, 2026 22:44
@codecov
Copy link

codecov bot commented Mar 15, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 29.49%. Comparing base (2281599) to head (baa33d3).
⚠️ Report is 341 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5679      +/-   ##
==========================================
+ Coverage   19.52%   29.49%   +9.96%     
==========================================
  Files          19       18       -1     
  Lines        3021     2417     -604     
==========================================
+ Hits          590      713     +123     
+ Misses       2310     1552     -758     
- Partials      121      152      +31     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@rene
Copy link
Contributor

rene commented Mar 16, 2026

Absolutely zero needs to run Eden tests on this PR. merging....

@rene rene merged commit 3ea3393 into lf-edge:master Mar 16, 2026
45 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants